Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CS2103-T13-2] CampusConnect #18

Open
wants to merge 573 commits into
base: master
Choose a base branch
from
Open

[CS2103-T13-2] CampusConnect #18

wants to merge 573 commits into from

Conversation

kokrui
Copy link

@kokrui kokrui commented Sep 14, 2023

CampusConnect, designed exclusively for NUS students living on campus, simplifies campus life. Keep track of shared expenses, never forget a friend's birthday, and easily access room numbers and campus services. Stay organised, stay connected, and make your on-campus experience at NUS stress-free with CampusConnect!

@codecov
Copy link

codecov bot commented Sep 14, 2023 •
edited
Loading

Codecov Report

Attention: 98 lines in your changes are missing coverage. Please review.

Comparison is base (517b785) 75.26% compared to head (91d9d9a) 83.02%.

Files Patch % Lines
.../java/seedu/address/model/util/SampleDataUtil.java 25.00% 12 Missing ⚠️
...main/java/seedu/address/ui/NotificationWindow.java 0.00% 12 Missing ⚠️
...va/seedu/address/logic/commands/AddAltCommand.java 88.50% 1 Missing and 9 partials ⚠️
...edu/address/logic/parser/FindExpressionParser.java 94.02% 4 Missing and 4 partials ⚠️
src/main/java/seedu/address/MainApp.java 0.00% 6 Missing ⚠️
src/main/java/seedu/address/ui/PersonCard.java 87.23% 6 Missing ⚠️
...edu/address/logic/commands/UpdatePhotoCommand.java 85.29% 2 Missing and 3 partials ⚠️
...n/java/seedu/address/model/event/EventFactory.java 80.00% 4 Missing and 1 partial ⚠️
...java/seedu/address/logic/commands/EditCommand.java 92.15% 0 Missing and 4 partials ⚠️
.../java/seedu/address/storage/JsonAdaptedAvatar.java 66.66% 3 Missing and 1 partial ⚠️
... and 16 more
Additional details and impacted files 
@@             Coverage Diff              @@
##             master      #18      +/-   ##
============================================
+ Coverage     75.26%   83.02%   +7.76%     
- Complexity      419      777     +358     
============================================
  Files            71      100      +29     
  Lines          1338     2380    +1042     
  Branches        126      267     +141     
============================================
+ Hits           1007     1976     +969     
- Misses          301      336      +35     
- Partials         30       68      +38

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

conradsoon and others added 29 commits November 1, 2023 11:00
@conradsoon @lordidiot
Update docs/UserGuide.md
e2e561b 
Co-authored-by: lordidiot <[email protected]>
@conradsoon
Add missing image
f7fcc2d
@conradsoon
Remove leftover comments
7bfad4a
@lordidiot
Merge pull request #126 from lordidiot/branch-dg
e6f340b 
Clean up errors in Developer Guide
@lordidiot
Update docs/UserGuide.md
d046001
@conradsoon
Merge pull request #127 from conradsoon/docs
1cbf5da 
Tweak UserGuide for `notes`, update emergency contacts feature.
@kokrui
Update UG yellow box definition
6b3c4f5
@kokrui
Merge pull request #132 from kokrui/ug-yellow-warning
cd1879c 
Update UG yellow box definition
@limjunxian1
Improve error specificity for addalt and edit command
ed11324
@limjunxian1
Fix test cases
cd91477
@limjunxian1
Merge pull request #130 from limjunxian1/1.3b-improvement
9a79599 
Update implementation, test cases and documentation for addalt and edit feature
@kokrui
Remove UG note section whitespace errors
5ba49d5
@lordidiot
Add bin to gitignore
733c1db
@kokrui
Merge pull request #133 from kokrui/ug-pay-new
e52e9bd 
Reorganize UG sections to new hierarchy
@lordidiot
Merge branch 'master' into ug-changes
d7cbaec
@lordidiot
Update user guide for notification system
2bb338f
@lordidiot
Fix missing nl
9c06650
@lordidiot
Remove leftover image
95b28f6
@lordidiot
Update headers and toc
e84a1cc
@lordidiot
Increase margin for User Guide
001d996
@conradsoon
Change sequence diagram for notes
63770f2
@conradsoon
Remove equal
7392c2a
@lordidiot
Update TOC with links
abdcfae
@conradsoon
Move constructor for PlantUML boxes, add parse, recolour boxes
1cf9c03
@conradsoon
Merge pull request #128 from conradsoon/docs-dg
87a5edf 
Update `notes` feature in DG to be more similar in style to other features in DG
@conradsoon
Change design of notes
3a93b79
@conradsoon
Replace example table with unordered list
4d7d36b
@lordidiot
Merge pull request #137 from lordidiot/branch-ug-toc
7b49239 
Update UG table of contents and spacing
@limjunxian1
Update UserGuide.md
55bad27
kokrui and others added 30 commits November 13, 2023 16:02
@kokrui
Fix more incorrect DG links
0b7bad3
@conradsoon
Undo changes to UG
62850c2
@kokrui
Merge pull request #237 from kokrui/pay-dg
a113040 
Update Payments feature DG section
@kokrui
Merge pull request #239 from kokrui/thanks-andrew
202e242 
Fix package name typo and outdated AB3 links in DG
@conradsoon
Merge pull request #235 from conradsoon/planned-enhancement
67e01fc 
Add planned enhancement for corrupt addressbook.json, move planned enhancements to DG
@lordidiot
Merge pull request #234 from lordidiot/update-ug
fd444bb 
Update User Guide
@conradsoon
Add effort appendix section to DG
b5702e6
@kokrui
Add new DG content for find
f60ae1a
@limjunxian1
Remove aarch64 javafx from gradle
1874366
@limjunxian1
Merge pull request #244 from limjunxian1/1.4-build
2bc9b46 
Remove aarch64 javafx from gradle
@limjunxian1
Fix error in diagram
d251f02
@limjunxian1
Update sequence diagram
7f38c91
@limjunxian1
Merge pull request #245 from limjunxian1/1.4-build
0210e7b 
Fix error in activity and sequence diagram for DG
@conradsoon
Fix error with DG
917f91a
@conradsoon
Merge pull request #240 from conradsoon/planned-enhancement
b72be79 
Add effort appendix section to DG
@lordidiot
Update updatephoto DG
e5051e7
@kokrui
Update find sequence diagram
b5657b3
@lordidiot
Merge pull request #246 from lordidiot/fix-updatephoto-dg
b6c6a85 
Update updatephoto DG
@kokrui
Merge pull request #243 from kokrui/new-find-dg
90ca2c9 
Add new DG content for Find command
@lordidiot
Update UG pagebreaks
ef4a73f
@lordidiot
Merge pull request #247 from lordidiot/pdf-ug
8170e43 
Update UG pagebreaks
@kokrui
Add missing line break in FAQ UG
458d400
@kokrui
Merge pull request #248 from kokrui/missing-br
c438ec9 
Add missing line break in FAQ UG
@lordidiot
Change h2 color for CS2101
ab5bc37
@lordidiot
Merge pull request #249 from lordidiot/h2-color
8db6dc0 
[CS2101] Change H2 color in UG
@lordidiot
Fix typo
9e42113
@lordidiot
Merge pull request #250 from lordidiot/fix-typo
f468780 
[CS2101] Fix typo
@lordidiot
Fix links
b86fdc8
@lordidiot
Remove reference to missing planned enhancements
8da204b
@lordidiot
Merge pull request #251 from lordidiot/fix-links-ug
91d9d9a 
[CS2101] Fix links
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kokrui @conradsoon @lordidiot @limjunxian1 @VN-Hao